Skip to content

chore: remove the ekvilibro-testnet deploy stage - #404

Merged
luislhl merged 1 commit into
mainfrom
chore/drop-ekvilibro-testnet
Aug 25, 2026
Merged

chore: remove the ekvilibro-testnet deploy stage#404
luislhl merged 1 commit into
mainfrom
chore/drop-ekvilibro-testnet

Conversation

@luislhl

@luislhl luislhl commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Removes the ekvilibro-testnet deploy stage. The network is being decommissioned in
ops-tools#1517 — it has no users, and
teardown is approved.

Why this needs to land before the teardown

deploy-ekvilibro-testnet is gated on needs.init.outputs.environment == 'testnet', and every
v*.*.*-rc* tag
sets that. Until this merges, any release candidate redeploys the whole serverless
stack — API Gateway, ~40 Lambdas, CloudWatch log groups — into an account the decommission is
emptying.

Merging early is safe: the network has no users, and its wallet-service served zero user requests in
the 30 days measured in
ops-tools#1481.

⚠️ Stage naming

The two ekvilibro jobs do not use the network name as the serverless stage:

Job serverless_stage
deploy-ekvilibro-testnet (removed) ekvilibro
deploy-ekvilibro-mainnet (kept) ekvi-main

So the bare stage ekvilibro is testnet. Worth knowing when reading resource names in that
account — anything matching hathor-explorer-service-ekvilibro-* is testnet, and
*-ekvi-main-* is mainnet.

What changed

  • .github/workflows/deploy.yml — deleted the deploy-ekvilibro-testnet job

What is deliberately untouched

deploy-ekvilibro-mainnet targets the same AWS account and is gated on
environment == 'mainnet'. Unchanged.

Verification

  • No ekvilibro-testnet reference remains under .github/
  • All 11 ekvilibro-mainnet references preserved
  • deploy.yml parses; deploy-ekvilibro-mainnet still present with its mainnet gate intact

Note: PR #403 removes the testnet-playground stage from this same file. The hunks are separate,
so the two merge independently.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WGPsyUjXZ9xD1QaMjExb6v

Summary by CodeRabbit

  • Chores
    • Removed the automated deployment workflow for the Ekvilibro testnet environment.
    • Testnet deployments and related health checks, notifications, and infrastructure configuration are no longer run by the release pipeline.

The ekvilibro-testnet network is being decommissioned in
HathorNetwork/ops-tools#1517. Until this lands, any v*-rc* tag redeploys the
whole serverless stack to that network, because the job is gated on
environment == 'testnet' and every release candidate satisfies that.

Note the stage name: ekvilibro-testnet deploys to serverless stage
"ekvilibro", while ekvilibro-mainnet deploys to "ekvi-main". Only the
testnet job is removed. deploy-ekvilibro-mainnet is gated on
environment == 'mainnet' and is unchanged.

Refs HathorNetwork/ops-tools#1517

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WGPsyUjXZ9xD1QaMjExb6v
@luislhl
luislhl requested a review from r4mmer as a code owner August 10, 2026 21:33
@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: b77deff9-d0a2-4d18-b5bc-1528adc21644

📥 Commits

Reviewing files that changed from the base of the PR and between a3f98bc and 592d719.

📒 Files selected for processing (1)
  • .github/workflows/deploy.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/deploy.yml

📝 Walkthrough

Walkthrough

The deployment workflow removes the deploy-ekvilibro-testnet job and its environment-specific deployment, infrastructure, health-check, notification, and secret configuration.

Changes

Deployment workflow

Layer / File(s) Summary
Remove Ekvilibro testnet deployment job
.github/workflows/deploy.yml
Removes the deploy-ekvilibro-testnet job and its reusable deployment configuration, including Ekvilibro-specific infrastructure settings and Slack secret wiring.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested reviewers: r4mmer

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the removal of the Ekvilibro testnet deployment stage.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/drop-ekvilibro-testnet

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@luislhl luislhl self-assigned this Aug 11, 2026
@luislhl luislhl moved this from Todo to In Progress (Done) in Hathor Network Aug 11, 2026
@luislhl luislhl moved this from In Progress (Done) to In Review (WIP) in Hathor Network Aug 24, 2026
@tuliomir tuliomir moved this from In Review (WIP) to In Review (Done) in Hathor Network Aug 24, 2026
@luislhl
luislhl merged commit 20946d4 into main Aug 25, 2026
5 checks passed
@github-project-automation github-project-automation Bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Aug 25, 2026
@luislhl
luislhl deleted the chore/drop-ekvilibro-testnet branch August 25, 2026 00:23
@luislhl luislhl moved this from Waiting to be deployed to Done in Hathor Network Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants